This repository was archived by the owner on Aug 30, 2022. It is now read-only.
alternative aggregation PoC #603
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Warning. The changes here are meant just for illustrative PoC purposes, and should not be merged.
This branch contains some test code to prove that it is possible, in principle, to support an alternative aggregation to "federated averaging" in the PET protocol. The particular example illustrated here is a histogram aggregation. A quick breakdown of the experiment:
0 - 5,5 - 10,10 - 15,15 - 20.5 - 10range. To convey this range to the coordinator, it would construct a "model"[0, 1, 0, 0].In a particular test run of this, spinning up a coordinator and running the test-drive with
-n 10, I observed[1, 0, 0, 0][0, 1, 0, 0][0, 0, 0, 1]On the coordinator, the unmasked model and histogram is visible in the console output:
Further remarks.
hist. While the above output looks sensible, my mileage varied a lot! it behaves a little peculiarly, sometimes adding / removing a point. It's not documented so perhaps I'm misusing it.